Skip to content

Conversation

@Menglths
Copy link

@Menglths Menglths commented Oct 20, 2025

Purpose

Performance testing tool based on the PyTest testing framework

Modifications

1、Added tests for UC-related performance metrics, including full throughput and incremental throughput.
2、Support for custom PC hit rate.
3、Support for custom tokenizer.

Test

image

case_hit_rate_map — {case_idx: hit_rate} 的映射
"""
print(f"[INFO] 共计 {len(test_cases)} 个测试用例待执行")
failed_case = []

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

failed_case is not used in this function.

@ygwpz
Copy link
Contributor

ygwpz commented Oct 23, 2025

put this file in benchmark dir seems better

@yuanzhg078
Copy link

Pursuant to the UCM code-repository guidelines, all code comments must be composed in English.

Copy link
Contributor

@Potterluo Potterluo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

log,config,single

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to add the pip packages you use along with their versions to make it easier for others who don't have them (e.g., pandas, pydantic) to use them.

server_url: "http://141.111.32.70:9382"
tokenizer_path: "/home/models/QwQ-32B"
# Performance Test Configuration
llmperf_test_cases:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Configuration items can be added, referring to the implementation of logs and reports, with results stored using timestamps. (They can be uniformly placed in the reports directory to prevent too many subdirectories, and there should be an llmperf flag.)
  2. Parameter names such as max_num_completed_requests and num_concurrent_requests are not descriptive enough; additional descriptions should be added.

from common.llmperf.utils.utils import reset_prefill_cache


def run_test_cases(test_cases, timestamp_dir, model, server_url, tokenizer_path):

This comment was marked as resolved.

@Menglths Menglths changed the title Added performance test [Feature] Added performance testing tool based on the PyTest testing framework Nov 5, 2025
@Menglths Menglths force-pushed the develop branch 2 times, most recently from 55e817f to f01918a Compare November 14, 2025 02:14
Copy link

@Wwwzff Wwwzff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we may pull the latest code from main branch and rebase your commit to the top

@Menglths Menglths force-pushed the develop branch 2 times, most recently from 3bd5375 to ebf4639 Compare November 14, 2025 04:58
@Menglths
Copy link
Author

we may pull the latest code from main branch and rebase your commit to the top

All commits have been rebased.

@Menglths Menglths force-pushed the develop branch 2 times, most recently from 21a07b1 to 8a90e76 Compare November 14, 2025 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants